Skip to content

Remove remaining Trace Explorer implementation#262265

Merged
rmyz merged 7 commits intoelastic:mainfrom
sbelastic:254449-remove-remaining-trace-explorer-implementation
Apr 14, 2026
Merged

Remove remaining Trace Explorer implementation#262265
rmyz merged 7 commits intoelastic:mainfrom
sbelastic:254449-remove-remaining-trace-explorer-implementation

Conversation

@sbelastic
Copy link
Copy Markdown
Contributor

@sbelastic sbelastic commented Apr 9, 2026

Doesn't close #254449 (missing doc updates)

Summary

  • Removes all remaining Trace Explorer implementation

Checklist

@sbelastic sbelastic self-assigned this Apr 9, 2026
@sbelastic sbelastic added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation labels Apr 9, 2026
@sbelastic
Copy link
Copy Markdown
Contributor Author

/ci

@sbelastic
Copy link
Copy Markdown
Contributor Author

/ci

@sbelastic
Copy link
Copy Markdown
Contributor Author

/ci

@sbelastic
Copy link
Copy Markdown
Contributor Author

/ci

@sbelastic
Copy link
Copy Markdown
Contributor Author

/ci

@sbelastic sbelastic marked this pull request as ready for review April 10, 2026 07:41
@sbelastic sbelastic requested review from a team as code owners April 10, 2026 07:41
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-presentation-team (Team:obs-presentation)

@macroscopeapp
Copy link
Copy Markdown
Contributor

macroscopeapp Bot commented Apr 10, 2026

Approvability

Verdict: Needs human review

This PR removes an entire user-facing feature (Trace Explorer) including routes, components, API endpoints, and tests. The author does not own any of the modified files, which are owned by multiple teams (@elastic/obs-presentation-team, @elastic/kibana-localization, @elastic/security-generative-ai). Feature removal of this scope requires review by the designated code owners.

You can customize Macroscope's approvability policy. Learn more.

Copy link
Copy Markdown
Contributor

@MiriamAparicio MiriamAparicio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for the cleaning up

Copy link
Copy Markdown
Contributor

@rmyz rmyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, maybe it makes sense to backport:all-open, but not sure as it includes security changes, we could just backport it to 9.3 as well

@rmyz rmyz added backport:version Backport to applied version labels v9.4.0 v9.3.4 and removed backport:skip This PR does not require backporting labels Apr 14, 2026
@rmyz rmyz enabled auto-merge (squash) April 14, 2026 14:55
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
apm 2130 2123 -7

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 2.7MB 2.7MB -11.3KB
securitySolution 11.7MB 11.7MB -868.0B
total -12.1KB

Public APIs missing exports

Total count of every type that is part of your API that should be exported but is not. This will cause broken links in the API documentation system. Target amount is 0. Run node scripts/build_api_docs --plugin [yourplugin] --stats exports for more detailed information.

id before after diff
apm 121 119 -2
Unknown metric groups

async chunk count

id before after diff
apm 85 83 -2

History

cc @sbelastic

@rmyz rmyz merged commit 62e265a into elastic:main Apr 14, 2026
24 checks passed
@kibanamachine
Copy link
Copy Markdown
Contributor

Starting backport for target branches: 9.3, 9.4

https://github.com/elastic/kibana/actions/runs/24410128815

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Apr 14, 2026
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
(cherry picked from commit 62e265a)
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 Some backports could not be created

Status Branch Result
9.3 Backport failed because of merge conflicts
9.4

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 262265

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Apr 14, 2026
# Backport

This will backport the following commits from `main` to `9.4`:
- [Remove remaining Trace Explorer implementation
(#262265)](#262265)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Samuel
Brito","email":"samuel.brito@elastic.co"},"sourceCommit":{"committedDate":"2026-04-14T16:15:02Z","message":"Remove
remaining Trace Explorer implementation (#262265)\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"62e265a1f5aba91a6c95b5cdaca3156927dbe235","branchLabelMapping":{"^v9.5.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport:version","v9.4.0","Team:obs-presentation","v9.5.0","v9.3.4"],"title":"Remove
remaining Trace Explorer
implementation","number":262265,"url":"https://github.com/elastic/kibana/pull/262265","mergeCommit":{"message":"Remove
remaining Trace Explorer implementation (#262265)\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"62e265a1f5aba91a6c95b5cdaca3156927dbe235"}},"sourceBranch":"main","suggestedTargetBranches":["9.4","9.3"],"targetPullRequestStates":[{"branch":"9.4","label":"v9.4.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v9.5.0","branchLabelMappingKey":"^v9.5.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/262265","number":262265,"mergeCommit":{"message":"Remove
remaining Trace Explorer implementation (#262265)\n\nCo-authored-by:
Sergi Romeu
<sergi.romeu@elastic.co>","sha":"62e265a1f5aba91a6c95b5cdaca3156927dbe235"}},{"branch":"9.3","label":"v9.3.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Samuel Brito <samuel.brito@elastic.co>
Co-authored-by: Sergi Romeu <sergi.romeu@elastic.co>
@rmyz rmyz removed the v9.3.4 label Apr 15, 2026
@rmyz
Copy link
Copy Markdown
Contributor

rmyz commented Apr 15, 2026

Removing the v9.3.4 backport label, 9.3 has too many merge conflicts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:obs-presentation Focus: APM UI, Infra UI, Hosts UI, Universal Profiling, Obs Overview and left Navigation v9.4.0 v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[APM] Remove remaining Trace Explorer implementation

6 participants